Skip to content

Add star history to README#4192

Closed
VictoriousRaptor wants to merge 1 commit into
devfrom
VictoriousRaptor-patch-1
Closed

Add star history to README#4192
VictoriousRaptor wants to merge 1 commit into
devfrom
VictoriousRaptor-patch-1

Conversation

@VictoriousRaptor
Copy link
Copy Markdown
Contributor

image

@VictoriousRaptor VictoriousRaptor self-assigned this Dec 30, 2025
@VictoriousRaptor VictoriousRaptor added the Documentation Update required to documentation label Dec 30, 2025
@github-actions github-actions Bot added this to the 2.1.0 milestone Dec 30, 2025
@gitstream-cm
Copy link
Copy Markdown

gitstream-cm Bot commented Dec 30, 2025

🥷 Code experts: Jack251970, jjw24

Jack251970, jjw24 have most 👩‍💻 activity in the files.
Jack251970 has most 🧠 knowledge in the files.

See details

README.md

Activity based on git-commit:

Jack251970 jjw24
DEC 4 additions & 4 deletions
NOV 1 additions & 0 deletions
OCT 11 additions & 3 deletions
SEP 2 additions & 1 deletions 10 additions & 9 deletions
AUG 16 additions & 9 deletions
JUL

Knowledge based on git-blame:
Jack251970: 92%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

@gitstream-cm
Copy link
Copy Markdown

gitstream-cm Bot commented Dec 30, 2025

Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX.

@coderabbitai coderabbitai Bot added the enhancement New feature or request label Dec 30, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 30, 2025

📝 Walkthrough

Walkthrough

Adds a "Star History" section with a chart badge and link to the Flow.Launcher repository in README.md. The new section is inserted at two locations within the document and includes a visual history chart without modifying existing content.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added "Star History" section with chart badge/link in two locations; no existing content altered

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

enhancement

Suggested reviewers

  • Garulf
  • JohnTheGr8
  • Yusyuriv

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description contains only an image attachment without any textual explanation of the changes, making it difficult to understand the purpose or context of the pull request. Add a brief textual description explaining what the star history section is, why it's being added, and what value it provides to the repository.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add star history to README' directly and clearly summarizes the main change in the pull request, which adds a Star History section with chart badge/link to the README.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
README.md (1)

386-390: Add a divider line after Star History to maintain document formatting consistency.

The Star History section lacks a separator before the Development section. Other major sections in the README (e.g., "Questions/Suggestions" → "Development" at lines 379–390) are separated by the divider image. Add one after line 389 to align with the established pattern.

🔎 Proposed formatting adjustment
 [![Star History Chart](https://api.star-history.com/svg?repos=Flow-Launcher/Flow.Launcher&type=date&legend=top-left)](https://www.star-history.com/#Flow-Launcher/Flow.Launcher&type=date&legend=top-left)
+
+<img src="https://user-images.githubusercontent.com/6903107/144858082-8b654daf-60fb-4ee6-89b2-6183b73510d1.png" width="100%">
+
 ## Development
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8092a44 and 0865194.

📒 Files selected for processing (1)
  • README.md
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2024-10-08T15:52:58.573Z
Learnt from: taooceros
Repo: Flow-Launcher/Flow.Launcher PR: 2616
File: Flow.Launcher/Flow.Launcher.csproj:7-7
Timestamp: 2024-10-08T15:52:58.573Z
Learning: In the Flow Launcher project, the version number in the `Flow.Launcher.csproj` file is dynamically updated during the CI/CD process.

Applied to files:

  • README.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: gitStream.cm
  • GitHub Check: build
🔇 Additional comments (2)
README.md (2)

386-389: Inconsistency: AI summary mentions "two locations" but only one Star History section is visible.

The summary states the changes are "inserted at two locations within the document," but the provided code shows only one Star History section at lines 386-389. Please clarify if there's a second location or if the summary is inaccurate.


388-388: Both star-history.com URLs are accessible and stable.

Verified that the badge image endpoint and web destination both return HTTP 200 status codes, confirming the service is operational and the parameters (repo path, date type, legend position) are working as intended.

@VictoriousRaptor
Copy link
Copy Markdown
Contributor Author

close as we've discussed

@VictoriousRaptor VictoriousRaptor deleted the VictoriousRaptor-patch-1 branch January 3, 2026 10:15
@jjw24 jjw24 removed this from the 2.1.0 milestone Jan 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Update required to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants